Skip to content

feat(art): add art funny-cake#22

Merged
mr150 merged 4 commits into
mainfrom
feat/63842047_art-cake
May 7, 2026
Merged

feat(art): add art funny-cake#22
mr150 merged 4 commits into
mainfrom
feat/63842047_art-cake

Conversation

@oostap1985
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread src/arts/funny-cake.ejs Outdated
<div class="Ps-a B10p L0 W100p H20p Bdrd50p Bgc-$cakeGreen">
<!-- Beads -->
<div class="Ps-a T50p L0 -Sz100p">
<div class="<%= css.bead %> T-10p L1p W6p"></div>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Может попробовать как-то в цикле вывести эти элементы, чтобы меньше бойлерплейта было? Вроде есть паттерн по расположению и размеру

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Исправил. Я даже и не заметил. Круто.

Comment thread src/arts/funny-cake.ejs Outdated
</div>
</div>
</div>
<div class="<%= css.creamY %> T15p L72p W30p">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Эту часть тоже можно в цикле сделать

Comment thread src/arts/funny-cake.ejs
creamY: 'Ps-a Apcr2/1 Bdrd50p Bgc-$cakeYellow Ov-h'
creamWhite: 'Ps-a -Sz100p Bdrd50p Bgc-$cakeWhite',
creamYellow: 'Ps-a Apcr2/1 Bdrd50p Bgc-$cakeYellow Ov-h',
leftBeads: ['T-10p L1p W6p', 'T0p L3p W8p', 'T16p L7p W10p', 'T28p L13p W12p', 'T38p L22p W15p', 'T45p L34p W18p'],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Массивы с классами лучше в отдельный словарь выносить

@mr150 mr150 merged commit 15be461 into main May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants